Feature/home widgets#10
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
i3months
reviewed
May 13, 2026
| @@ -0,0 +1,52 @@ | |||
| import { useEffect, useState } from 'react' | |||
Member
There was a problem hiding this comment.
텍스트 한글자씩 나타나는 애니메이션 만들때 쓰는 코드인가요 좋네요
| let intervalId: number | undefined | ||
| const timeoutId = window.setTimeout(() => { | ||
| let i = 0 | ||
| intervalId = window.setInterval(() => { |
Member
There was a problem hiding this comment.
setInterval 대신 setTimeout 이 더 나을수 있겠네요.
탭이 백그라운드에서 실행될 때 콜백이 딜레이 될 수 있을듯합니다.
Contributor
Author
There was a problem hiding this comment.
좋은 리뷰 감사합니다. 제가 또 따로 찾아보니 requestAnimationFrame 이란걸 쓰면 좋다고도 하네요. 한번 찾아보고 수정하겠습니다.
Member
|
👍 |
csh1668
added a commit
to csh1668/stackup
that referenced
this pull request
May 13, 2026
…gets Feature/home widgets
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 작업 목적
홈페이지를 구성하는 메인 위젯(섹션)들과 이를 보조하는 UI 컴포넌트들을 구현합니다.
✨ 주요 변경 사항
HomeHero: 상단 히어로 섹션HomeServices: 서비스 소개 섹션HomeQuote: 인용구 섹션HomeFaq: 자주 묻는 질문(FAQ) 섹션HomeCta: 하단 CTA (Get Started) 섹션Laptop,ScreenContent컴포넌트 구현📚 PR 구조 및 분할 사유 (Reviewer Note)
저희 팀 Docs 컨벤션에 명시된 PR 변경 라인 수 제한을 준수하기 위해 작업을 단계별로 분할(Stacked PR)하여 올리고 있습니다.
초기 프로젝트 세팅이나 글로벌 설정의 경우 불가피하게 변경 사항이 방대해질 수 있지만, 실제 기능 구현부터는 리뷰어분들의 편의와 리뷰 피로도 감소를 위해 아래와 같이 논리적인 단위로 PR을 나누었습니다.
feature/hompage(설정 및 문서 변경)feature/home-layout(전역 헤더/푸터 레이아웃)feature/home-widgets(홈페이지 상세 섹션 위젯)본 PR은 두 번째 PR인
feature/home-layout을 기반(Base)으로 하고 있습니다. 이전 레이아웃 작업분은 제외되고 순수하게 **'추가된 홈페이지 위젯 코드'**만 표시되니, 이 부분만 편하게 리뷰해 주시면 됩니다!